Skip to content

feat: split API overview page out of MdxRoute into ApiOverviewRoute#1226

Merged
jderochervlk merged 1 commit intomasterfrom
vlk/split-out-api-overview
Apr 12, 2026
Merged

feat: split API overview page out of MdxRoute into ApiOverviewRoute#1226
jderochervlk merged 1 commit intomasterfrom
vlk/split-out-api-overview

Conversation

@jderochervlk
Copy link
Copy Markdown
Collaborator

@jderochervlk jderochervlk commented Apr 3, 2026

  • Create ApiOverviewRoute.res for /docs/manual/api with ApiOverviewLayout.Docs
  • Add explicit route in routes.res pointing docs/manual/api to ApiOverviewRoute
  • Remove API overview branch, manualTableOfContents, and sidebar helpers from MdxRoute
  • MdxRoute now only handles docs/manual and docs/react pages via react-router-mdx

Note

This work was done with AI assistance

@jderochervlk jderochervlk force-pushed the vlk/split-out-api-overview branch from 4412018 to 8fda350 Compare April 9, 2026 13:36
@jderochervlk jderochervlk changed the base branch from vlk/split-out-syntax-lookup to master April 9, 2026 13:36
@jderochervlk jderochervlk marked this pull request as ready for review April 9, 2026 13:36
jderochervlk added a commit that referenced this pull request Apr 9, 2026
* ci: allow dependabot PRs to skip deployments to cloudflare (#1235)

* ci: allow dependabot PRs to deploy via pull_request_target

GitHub restricts secrets for pull_request events triggered by
dependabot[bot]. Switch dependabot PRs to pull_request_target,
which runs in the base branch context and has access to secrets.

- Add pull_request_target trigger
- Route dependabot PRs through pull_request_target only
- Route all other PRs through pull_request only (no double runs)
- Checkout PR head SHA for pull_request_target events

* Update .github/workflows/deploy.yml

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* bypass cloudflare

* Simplify deploy job condition in workflow file

* Fix conditional syntax in deploy workflow steps

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* fix: patch marked ReDoS vulnerability (CVE-2022-21681) via Yarn resolution override (#1236)

* Initial plan

* fix: upgrade marked to 4.0.10 via resolutions to fix ReDoS (GHSA-5v2h-r2cx-5xgj)

Agent-Logs-Url: https://github.com/rescript-lang/rescript-lang.org/sessions/9eb986e6-cb64-40d5-ac83-ff5bdd72d561

Co-authored-by: jderochervlk <60623931+jderochervlk@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: jderochervlk <60623931+jderochervlk@users.noreply.github.com>

* feat: split community pages out of MdxRoute into CommunityRoute (#1223)

- Create CommunityRoute.res with dedicated loader and community sidebar
- Register communityRoutes in routes.res, filter community from mdxRoutes
- Remove communityTableOfContents, community branches from MdxRoute

* fix: Add .resi file for CommunityRoute.jsx (#1238)

* Add Cypress E2E testing with ReScript bindings and CI integration (#1239)

* Add Cypress E2E testing with ReScript bindings and CI integration

- Add Cypress config and support files for E2E tests
- Add ReScript bindings for Cypress in e2e/bindings
- Add navigation E2E test in e2e/Navigation_.cy.res
- Update .gitignore for e2e artifacts
- Add Cypress and E2E scripts to package.json
- Add e2e to rescript.json dev sources
- Update GitHub Actions to run E2E tests after deploy

* Use Cypress GitHub Action for E2E tests in deploy workflow

* pr feedback

* wait again

* configure retries

* change type name

* Initial plan

---------

Co-authored-by: Josh Vlk <josh@vlkpack.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: jderochervlk <60623931+jderochervlk@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown

Cloudflare deployment

Deployement ID: dc33050b-6ece-424b-8233-da39438af2ae
Deployment Environment: preview

⛅️ wrangler 4.63.0 (update available 4.81.1)
─────────────────────────────────────────────
✨ Compiled Worker successfully
Uploading... (7841/7843)
Uploading... (7842/7843)
Uploading... (7843/7843)
✨ Success! Uploaded 2 files (7841 already uploaded) (1.53 sec)

✨ Uploading _redirects
✨ Uploading Functions bundle
🌎 Deploying...
✨ Deployment complete! Take a peek over at https://dc33050b.rescript-lang.pages.dev
✨ Deployment alias URL: https://vlk-split-out-api-overview.rescript-lang.pages.dev

@jderochervlk jderochervlk merged commit 98addbf into master Apr 12, 2026
6 checks passed
@jderochervlk jderochervlk deleted the vlk/split-out-api-overview branch April 12, 2026 19:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants